-
Notifications
You must be signed in to change notification settings - Fork 17
New landing page design #119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Preview the changes: https://turinglang.org/pr-previews/119 |
If anyone is interested, then please feel free to take on this PR! |
* very basic redesign * make buttons interactive
* add dangling code example * add code example section
* quickfix alignment * add news and clean up scss
* improve colors * better link hover color
* paste in svg * responsive svg header
using the same background colour for the team "cards" as for the non-hoverable cards on the landing page
- added links to discourse and slack to the buttons at the top of the page - moved link to libraries page to "Learn more" section
Co-authored-by: Penelope Yong <[email protected]>
0046a57
to
e9e14d5
Compare
I will definitely have time to finish this in July. Which points would you like to see addressed before this PR can be merged? |
Revisiting this. Is there a reason why the old PR was closed? Also, I'd personally be happy to merge if some of the placeholder content is removed. Honestly, it's not like the existing front page has any resources, so even a fairly clean slate would be a huge improvement over the current one. |
I'm happy to take this on some time next week — I will try to get it into a mergeable state and we can always iterate further after it's merged. |
Apologies for the confusion. I don’t have admin permissions on this repo, and the previous PR had complex merge conflicts. Since it was from Simon’s fork, and I was adding some features to the landing page, I accidentally messed up some git operations. So I took those changes and opened a new PR from a branch within this repo to make it easier to manage.
Totally agree. However, before merging, I think we should either:
|
Continuing #105
Preview the changes: https://turinglang.org/pr-previews/119